Skip to content

Implement interim subevent check-in system - #377

Merged
samderanova merged 3 commits into
mainfrom
feature/subevent-checkin
Jan 27, 2024
Merged

Implement interim subevent check-in system#377
samderanova merged 3 commits into
mainfrom
feature/subevent-checkin

Conversation

@taesungh

@taesungh taesungh commented Jan 27, 2024

Copy link
Copy Markdown
Member

All temporary, as part of #359, add endpoints and interface to check-in participants to subevents. Still has a bug where the old badge number is automatically reinserted into input.

@taesungh
taesungh requested a review from samderanova January 27, 2024 03:01
Comment thread apps/api/src/admin/participant_manager.py Outdated
Comment thread apps/api/src/routers/admin.py Outdated
@github-actions

github-actions Bot commented Jan 27, 2024

Copy link
Copy Markdown
Contributor

Deploy preview for irvinehacks-site-2024 ready!

Name IrvineHacks 2024 Site
Preview Visit Preview
Commit ebfe028

@taesungh
taesungh force-pushed the feature/subevent-checkin branch from 69a4c65 to 2664295 Compare January 27, 2024 03:10
@taesungh
taesungh force-pushed the feature/subevent-checkin branch from 2664295 to fc30acf Compare January 27, 2024 03:11
log.info(f"Non-hacker {uid} status updated to attending by {director.uid}")


async def subevent_checkin(event_id: str, uid: str, organizer: User) -> None:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docstring, but can leave it for later.



@router.post("/event-checkin/{event}")
async def subevent_checkin(

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docstring

Comment thread apps/api/src/admin/participant_manager.py
Comment thread apps/site/src/app/admin/events/Events.tsx
@samderanova
samderanova merged commit 2263491 into main Jan 27, 2024
@IanWearsHat
IanWearsHat deleted the feature/subevent-checkin branch November 24, 2025 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants